Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConfigDrive support #24

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ConfigDrive support #24

wants to merge 2 commits into from

Conversation

aka-bo
Copy link

@aka-bo aka-bo commented Mar 6, 2016

Initial support for the ConfigDrive metadata service as
well as the NetworkConfig and UserData Plugins

Notes:

  • FreeBSDConfigDriveManager cdrom sources only (raw
    hdd and vfat is currently stubbed)
  • enabled freebsd.networkconfig.NetworkConfigPlugin
    and modified to play nice with the metadata services
  • fixed a filtering bug in FreeBSDUtils.get_network_adapters
  • enabled the UserDataPlugin - we're only using the
    user_data script today and more work will be needed
    before all user_data plugins are functional
  • plugins.common.execcmd.Bash modified to use the sh command
    for portability (bash isn't installed by default on freebsd)

Initial support for the ConfigDrive metadata service as
well as the NetworkConfig and UserData Plugins

Notes:
  - FreeBSDConfigDriveManager cdrom sources only (raw
    hdd and vfat is currently stubbed)
  - enabled freebsd.networkconfig.NetworkConfigPlugin
    and modified to play nice with the metadata services
  - fixed a filtering bug in FreeBSDUtils.get_network_adapters
  - enabled the UserDataPlugin - we're only using the
    user_data script today and more work will be needed
    before all user_data plugins are functional
  - plugins.common.execcmd.Bash modified to use the sh command
    for portability (bash isn't installed by default on freebsd)
@@ -1,4 +1,5 @@
pbr>=0.10
importlib
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The default version of python on freebsd is 2.7.

I think we do not need this.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed. I will remove this

@pellaeon
Copy link
Owner

pellaeon commented Aug 2, 2016

Hi, thanks for the pull request, I'm in the process of building up a Jenkins environment that can build test images (currently met with some difficulty). I'll test your PR once Jenkins has been set up. (It might take weeks, track the progress here.

@dpeschman
Copy link

Hello. Any news on this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants